Skip to content

fix(webapp): Truncate long branch names to prevent breaking the onboarding layout#2954

Merged
matt-aitken merged 1 commit intomainfrom
fix(webapp)-prevent-long-branch-names-wrapping
Jan 27, 2026
Merged

fix(webapp): Truncate long branch names to prevent breaking the onboarding layout#2954
matt-aitken merged 1 commit intomainfrom
fix(webapp)-prevent-long-branch-names-wrapping

Conversation

@samejr
Copy link
Member

@samejr samejr commented Jan 27, 2026 •
edited by devin-ai-integration bot
Loading

Before

CleanShot 2026-01-21 at 20 11 08@2x

After

CleanShot 2026-01-27 at 17 32 47@2x
Open with Devin

@changeset-bot
Copy link

changeset-bot bot commented Jan 27, 2026

⚠️ No Changeset found

Latest commit: c89f8c6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 27, 2026 •
edited
Loading

Walkthrough

The header area of the DeploymentOnboardingSteps component in BlankStatePanels.tsx is updated with Tailwind CSS utility classes to improve layout and overflow handling. Specifically, min-w-0 is added to an inner wrapper div to allow proper shrinking, shrink-0 is added to EnvironmentIcon to prevent shrinking, and truncate is added to the Header1 element to prevent text overflow. The package.json file shows 3 lines added and 3 lines removed, indicating a dependency or configuration update.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ⚠️ Warning The PR description is minimal, containing only before/after screenshots without following the required template structure. Add required sections: issue reference (Closes #), checklist items, testing steps, changelog description, and any additional context about the changes.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: truncating long branch names to prevent layout breaking in the onboarding component.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@samejr samejr marked this pull request as ready for review January 27, 2026 17:51
devin-ai-integration[bot]
devin-ai-integration bot reviewed Jan 27, 2026
View reviewed changes
Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional flags.

Open in Devin Review

nicktrn
nicktrn approved these changes Jan 27, 2026
View reviewed changes
Copy link
Collaborator

@nicktrn nicktrn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

…

@matt-aitken matt-aitken merged commit d4e4fbd into main Jan 27, 2026
34 checks passed
@matt-aitken matt-aitken deleted the fix(webapp)-prevent-long-branch-names-wrapping branch January 27, 2026 18:26
@vibe-kanban-cloud
Copy link

vibe-kanban-cloud bot commented Jan 28, 2026

Review Complete

Your review story is ready!

View Story

Comment !reviewfast on this PR to re-generate the story.

NERLOE pushed a commit to NERLOE/trigger.dev that referenced this pull request Jan 30, 2026
@samejr @NERLOE
fix(webapp): Truncate long branch names to prevent breaking the onboa…
b91ffee 
…rding layout (triggerdotdev#2954)

### Before
<img width="1736" height="964" alt="CleanShot 2026-01-21 at 20 11 08@2x"
src="https://github.com/user-attachments/assets/76c6ed42-d2a0-4212-aaad-6fb848025c85"
/>

### After
<img width="1776" height="1032" alt="CleanShot 2026-01-27 at 17 32
47@2x"
src="https://github.com/user-attachments/assets/030ba26e-6aad-4232-a9ff-f2fde9931f8c"
/>

<!-- devin-review-badge-begin -->

---

<a
href="https://app.devin.ai/review/triggerdotdev/trigger.dev/pull/2954">
  <picture>
<source media="(prefers-color-scheme: dark)"
srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1">
<img
src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1"
alt="Open with Devin">
  </picture>
</a>
<!-- devin-review-badge-end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments

@matt-aitken matt-aitken matt-aitken approved these changes

@nicktrn nicktrn nicktrn approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@samejr @matt-aitken @nicktrn